Skip to content

feat: add tech debt counter for disabled varHead simp warnings#21

Merged
bryangingechen merged 1 commit into
masterfrom
add-varHead-tech-debt-counter
Apr 28, 2026
Merged

feat: add tech debt counter for disabled varHead simp warnings#21
bryangingechen merged 1 commit into
masterfrom
add-varHead-tech-debt-counter

Conversation

@kim-em

@kim-em kim-em commented Apr 18, 2026

Copy link
Copy Markdown
Contributor

This PR adds a counter for set_option warning.simp.varHead false occurrences to the weekly tech debt report.

The warning.simp.varHead option silences a warning about simp lemmas whose LHS has a variable at its head. Places where this warning is silenced are tech debt we want to track over time.

Prompted by a review comment on leanprover-community/mathlib4-nightly-testing#209, which introduces several set_option warning.simp.varHead false in uses in Mathlib/ and Archive/.

🤖 Prepared with Claude Code

This PR adds a counter for `set_option warning.simp.varHead false`
occurrences to the weekly tech debt report.

The `warning.simp.varHead` option silences a warning introduced in Lean
about simp lemmas whose LHS has a metavariable as its head. Places where
this warning is silenced are tech debt we want to track over time.

Prompted by discussion on
leanprover-community/mathlib4-nightly-testing#209
which adds several `set_option warning.simp.varHead false in` uses.

🤖 Prepared with Claude Code

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@bryangingechen bryangingechen merged commit d7b053b into master Apr 28, 2026
@bryangingechen bryangingechen deleted the add-varHead-tech-debt-counter branch April 28, 2026 02:53
kim-em added a commit to kim-em/mathlib-ci that referenced this pull request May 22, 2026
…rover-community#21)

This PR adds a counter for `set_option warning.simp.varHead false`
occurrences to the weekly tech debt report.

The `warning.simp.varHead` option silences a warning introduced in Lean
about simp lemmas whose LHS has a metavariable as its head. Places where
this warning is silenced are tech debt we want to track over time.

Prompted by discussion on
leanprover-community/mathlib4-nightly-testing#209
which adds several `set_option warning.simp.varHead false in` uses.

🤖 Prepared with Claude Code

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants